Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix : 데이터베이스 유니크 중복 예외 처리 #555

Merged
merged 6 commits into from
Jul 8, 2024
Merged

fix : 데이터베이스 유니크 중복 예외 처리 #555

merged 6 commits into from
Jul 8, 2024

Conversation

GaBaljaintheroom
Copy link
Collaborator

@GaBaljaintheroom GaBaljaintheroom commented Jul 7, 2024 •
edited
Loading

작업 내용 (Content)

  • 데이터베이스 Unique 제약조건 예외 처리

링크 (Links)

기타 사항 (Etc)

  • 데이터베이스 Unique 제약조건 예외를 Application 단위에서 잡지를 못함 -> 트랜잭션이 이후 DB단에서 예외가 발생하기 때문
  • @transactional을 사용하지 않거나 바로 flush를 하여 예외를 잡는 방법 중 후자를 선택함

Merge 전 필요 작업 (Checklist before merge)

희망 리뷰 완료 일 (Expected due date)

202X. X. X. Wed

@GaBaljaintheroom GaBaljaintheroom added the fix 버그 수정 label Jul 7, 2024
@GaBaljaintheroom GaBaljaintheroom self-assigned this Jul 7, 2024
seokho-1116
seokho-1116 reviewed Jul 8, 2024
View reviewed changes
Copy link
Collaborator

@seokho-1116 seokho-1116 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿굿

backend/core/src/main/java/site/timecapsulearchive/core/global/error/ErrorCode.java Outdated Show resolved Hide resolved
@GaBaljaintheroom GaBaljaintheroom merged commit f97c1df into develop_back_core Jul 8, 2024
2 checks passed
@seokho-1116 seokho-1116 deleted the fix/unique_error-B-#549 branch July 25, 2024 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seokho-1116 seokho-1116 seokho-1116 approved these changes

Assignees

@GaBaljaintheroom GaBaljaintheroom

Labels
fix 버그 수정
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GaBaljaintheroom @seokho-1116